Skip to content

[#349] Replace hardcoded $PLOT with RESERVE_LABEL constant#359

Merged
realproject7 merged 1 commit intomainfrom
task/349-reserve-label
Mar 19, 2026
Merged

[#349] Replace hardcoded $PLOT with RESERVE_LABEL constant#359
realproject7 merged 1 commit intomainfrom
task/349-reserve-label

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Replaced hardcoded const reserveLabel = IS_TESTNET ? "WETH" : "$PLOT" with imported RESERVE_LABEL from lib/contracts/constants.ts in all 7 affected components
  • Removed unused IS_TESTNET imports where no longer needed
  • No logic changes — label now reads "PL_TEST" on mainnet (matching the constant updated in PR [#339] Update frontend + SDK constants for Base mainnet #342)

Fixes #349

Files changed

  • WriterTradingStats.tsx
  • DonateWidget.tsx
  • ClaimRoyalties.tsx
  • ReaderPortfolio.tsx
  • PriceChart.tsx
  • TradingWidget.tsx
  • StoryCardStats.tsx

Test plan

  • next build passes cleanly
  • Verify reserve label shows "PL_TEST" on mainnet, "WETH" on testnet

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The PR matches issue #349 and replaces the hardcoded reserve label with the shared RESERVE_LABEL constant in all seven affected components. The change is minimal and follows the existing constants pattern without introducing logic changes.

Findings

  • No correctness or design issues found in the submitted diff.

Decision

Approved on code review. lint-and-typecheck was still pending at the time of review, so merge should still respect the required GitHub checks.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Clean mechanical replacement across all 7 components — consistent, no logic changes, unused IS_TESTNET imports removed. Approved.

@realproject7 realproject7 merged commit 9320038 into main Mar 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Hardcoded $PLOT label in 7 components — use RESERVE_LABEL constant

2 participants